Skip to content

Fix Linux and Raspberry Pi variants displayed "Intel" in /downloads#695

Open
ogoregen wants to merge 1 commit into
processing:mainfrom
ogoregen:main
Open

Fix Linux and Raspberry Pi variants displayed "Intel" in /downloads#695
ogoregen wants to merge 1 commit into
processing:mainfrom
ogoregen:main

Conversation

@ogoregen

@ogoregen ogoregen commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Was caused by getBit() returning Intel for all package names containing x64. Also changed the label of the Windows variant from Intel to 64-bit as the former was misleading/incorrect.

@catilac catilac self-requested a review June 24, 2026 16:37
@netlify

netlify Bot commented Jun 24, 2026

Copy link
Copy Markdown

Deploy Preview for java-processing-faf822 ready!

Name Link
🔨 Latest commit f667685
🔍 Latest deploy log https://app.netlify.com/projects/java-processing-faf822/deploys/6a3c0432e92d39000776747b
😎 Deploy Preview https://deploy-preview-695--java-processing-faf822.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@SableRaf

Copy link
Copy Markdown
Collaborator

Flagging for review: should check how the new links get picked up by Fathom analytics.

@catilac

catilac commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Thanks for calling that out! I was about to tag you @SableRaf in regards to the visual elements. But sounds like it's good. I'll check the analytics

@catilac catilac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inside of src/templates/download.js there is tracking logic. We need to make sure we're correctly tracking. I will make a PR for this @ogoregen since I need to create some tracking codes.

After that gets merged into this branch, we can merge into main if all looks well.

@SableRaf

SableRaf commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Another thing is the info tooltip for each distribution could include an explanation so ppl can tell the difference between each build.

I was about to tag you @SableRaf in regards to the visual elements.

I've been thinking about this too. I'd suggest putting the Snap button at the top since that's the recommended install for beginners, and the other two buttons could use a secondary color. People who want the .deb or flatpak versions likely know what they need, but a beginner downloading the wrong package is likely to get stuck.

@ogoregen

Copy link
Copy Markdown
Contributor Author

Got it, @catilac.

I think Snap isn't the one we should highlight as it serves only Ubuntu users. Flatpak is the option that is both easy and that serves the most number of distros @SableRaf.

@SableRaf

SableRaf commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Does Flatpak take care of MIME file association, setting up the JAVA environment variable that the VS Code extension uses to locate Processing, and configure support for the pde:// URI schema?

I know that some distributions (like Mint) choose to disable snapd by default, but you can still re-enable it.

@catilac

catilac commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

@ogoregen @SableRaf -- would it make sense to move this discussion into another issue? it sounds less urgent than fixing these links. let's move this discussion into discord and we can distill it into a task we can assign to @ogoregen?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants